Skip to content
This repository was archived by the owner on Feb 16, 2024. It is now read-only.

[Merged by Bors] - Logging #187

Closed
siegfriedweber wants to merge 41 commits into from
Closed

[Merged by Bors] - Logging #187

siegfriedweber wants to merge 41 commits into from

Conversation

@siegfriedweber
Copy link
Member

@siegfriedweber siegfriedweber commented Dec 8, 2022 •
edited by sbernauer
Loading

Description

  • Show opensearch-dashboards service with stackablectl services list.
  • Add logging demo.

TODOs before merging:

  • demos/demos-v1.yaml: Replace https://github.com/stackabletech/stackablectl/logging/demos/logging/zookeeper.yaml with https://github.com/stackabletech/stackablectl/main/demos/logging/zookeeper.yaml.
  • stacks/stacks-v1.yaml: Replace https://github.com/stackabletech/stackablectl/logging/stacks/logging/vector-aggregator-discovery.yaml with https://github.com/stackabletech/stackablectl/main/stacks/logging/vector-aggregator-discovery.yaml.
  • demos/logging/zookeeper.yaml: Remove repo: docker.stackable.tech/sandbox/logging when Logging docker-images#268 is merged and the images are built and deployed.
  • releases/releases.yaml: Remove the file when [Merged by Bors] - Logging zookeeper-operator#588 is merged.
  • stacks/stacks-v1.yaml: Replace stackableRelease: 22.11-logging with stackableRelease: 22.11 when [Merged by Bors] - Logging zookeeper-operator#588 is merged.

Review Checklist

  • Code contains useful comments
  • (Integration-)Test cases added (or not applicable)
  • Documentation added (or not applicable)
  • Changelog updated (or not applicable)
  • Cargo.toml only contains references to git tags (not specific commits or branches)

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

@siegfriedweber
Add logging stack with OpenSearch and OpenSearch Dashboards
055491a
@siegfriedweber
Add Vector aggregator to logging stack
1283176
@siegfriedweber
Redact OpenSearch password if desired
92b5173
@siegfriedweber
Add discovery config map for the Vector aggregator
76cdb6f
@siegfriedweber
Use Stackable release 22.11 in the logging stack
f96516d
@siegfriedweber
Add logging demo and release
3601687
@siegfriedweber
Add logging demo to the documentation
742a1f3
@siegfriedweber
Set up OpenSearch Dashboard with a search view
a61ac0d
@siegfriedweber
Show logs endpoint for OpenSearch Dashboards
8f75c60
@siegfriedweber
Add columns pod and container to the log view in OpenSearch Dashboards
57261d1
@siegfriedweber
Add logging configuration to the logging demo
484f535
@siegfriedweber
Adapt log levels in the logging demo
a5a859c
@siegfriedweber
Update ZooKeeper image version in the logging demo
42063cd
@siegfriedweber
Extend the documentation for the logging demo
74e8521
@siegfriedweber
Set external URLs to the images in the documentation for the logging …
dcbd087 
…demo
@siegfriedweber
Set console log level threshold for the vector container
12da6c4
@siegfriedweber
Adapt log levels in the logging demo
b594ec0
@siegfriedweber
Update logging demo to the new image selection structure
469aa77
@siegfriedweber
Configure OpenSearch users in the logging stack and show extra infos …
3846cf4 
…for OpenSearch Dashboards provided by the service
@siegfriedweber
Update credentials in the Vector aggregator
ba5b266
@siegfriedweber
Rename the column "Extra infos" into "Info"
a544afb
@siegfriedweber
Improve documentation of the logging demo
d604307
@siegfriedweber
Improve the documentation of the logging demo
3af6fb1
@siegfriedweber
Use relative file paths in the logging demo documentation
0d077b5
@siegfriedweber
Increase the size of the overview picture of the logging demo
f1a4ade
@siegfriedweber
Use SVG image for the ZooKeeper logo
5178b92
@siegfriedweber
Add the logging demo to the navigation
6db3677
@siegfriedweber
Fix arrows in overview image
ae5688a
@siegfriedweber
Fix navigation link
0d9bff4
@siegfriedweber
Fix typo
b58f8d0
@sbernauer sbernauer self-assigned this Dec 12, 2022
sbernauer
sbernauer reviewed Dec 12, 2022
View reviewed changes
@sbernauer
Copy link
Member

sbernauer commented Dec 12, 2022

Demo worked for me as well

@sbernauer
Copy link
Member

sbernauer commented Dec 12, 2022 •
edited
Loading

image
my usual problems with elastic/Opensource (ordering of lines) 🙃 But nothing we can do here afaik

On the good side: I force killed zk-server-0 so log collection worked well 👍

sbernauer
sbernauer approved these changes Dec 13, 2022
View reviewed changes
Copy link
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please wait for Felix

@fhennig fhennig mentioned this pull request Dec 13, 2022
Merged
fhennig
fhennig reviewed Dec 13, 2022
View reviewed changes
fhennig
fhennig approved these changes Dec 19, 2022
View reviewed changes
Copy link
Contributor

@fhennig fhennig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm happy now 😃 LGTM!

sbernauer
sbernauer approved these changes Dec 19, 2022
View reviewed changes
Copy link
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy as well

@siegfriedweber
Copy link
Member Author

siegfriedweber commented Dec 19, 2022

bors merge

bors bot pushed a commit that referenced this pull request Dec 19, 2022
@siegfriedweber
Logging (#187)
c7d68c5 
## Description

- Show opensearch-dashboards service with `stackablectl services list`.
- Add logging demo.
@bors
Copy link

bors bot commented Dec 19, 2022

Pull request successfully merged into main.

Build succeeded:

@bors bors bot changed the title Logging [Merged by Bors] - Logging Dec 19, 2022
@bors bors bot closed this Dec 19, 2022
@bors bors bot deleted the logging branch December 19, 2022 14:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers

@fhennig fhennig fhennig approved these changes

@sbernauer sbernauer sbernauer approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@fhennig fhennig

@siegfriedweber siegfriedweber

@sbernauer sbernauer

Labels

release/2023-01 type/feature-new

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@siegfriedweber @sbernauer @fhennig @lfrancke